Skip to content

Conversation

@cthielen
Copy link
Contributor

This PR introduces:

  • NotificationCenter.Message API
  • NotificationCenter (required)

It does not introduce:

  • Notification, NSNotification
  • NotificationCenter methods designed for Notification, e.g. post(:Notification) or addObserver(forName:object:queue:block)

These excluded types can be found in swift-corelibs-foundation.

To support swift-corelibs-foundation functionality, this PR exports a handful of methods and types as @_spi(SwiftCorelibsFoundation) public.

@cthielen
Copy link
Contributor Author

@swift-ci please test

@cthielen
Copy link
Contributor Author

@swift-ci please test

@cthielen
Copy link
Contributor Author

@swift-ci please test

1 similar comment
@cthielen
Copy link
Contributor Author

@swift-ci please test

@cthielen cthielen force-pushed the pr/notification-center-message branch from b9bebed to 7eb8345 Compare July 24, 2025 18:23
@cthielen
Copy link
Contributor Author

@swift-ci please test

@cthielen cthielen force-pushed the pr/notification-center-message branch from b8e0537 to a0891d3 Compare July 24, 2025 20:54
@cthielen
Copy link
Contributor Author

@swift-ci please test

@cthielen cthielen force-pushed the pr/notification-center-message branch from a0891d3 to 3c30830 Compare July 24, 2025 23:49
@cthielen
Copy link
Contributor Author

@swift-ci please test

Copy link
Contributor

@jmschonfeld jmschonfeld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! (assuming the swift-collections tag bump in the swift repo lands before this is merged)

@cthielen
Copy link
Contributor Author

@swift-ci please test

@cthielen
Copy link
Contributor Author

cthielen commented Aug 1, 2025

@swift-ci please test

@cthielen cthielen merged commit c37addb into swiftlang:main Aug 5, 2025
16 checks passed
@cthielen cthielen deleted the pr/notification-center-message branch August 5, 2025 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants